-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip building cmssw in case of runtime image when testing a new image #200
Conversation
A new Pull Request was created by @aandvalenzuela (Andrea Valenzuela) for branch master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
@aandvalenzuela , looks like the runtime images are really only |
thinking about it again, if we want runtime images to also support installation then we have to add all the packages which we have in bootstrap seed (which are a lot). I would suggest to add a new test for runtime images which actually use cmssw installation from cvmfs i.e. the test should not do any local cmssw installation but should use releases from cvmfs , create dev area and run short matrix and add on tests |
Pull request #200 was updated. |
I have tested the script for the runtime image at https://cmssdt.cern.ch/jenkins/job/test-cmssw-images/19/ and for a regular image at https://cmssdt.cern.ch/jenkins/job/test-cmssw-images/20/ (I will delete the job once this PR is merged since this script will run on the regular https://cmssdt.cern.ch/jenkins/job/cms-containers-run-cmssw-test/ job) |
tests looks good |
Pull request #200 was updated. |
Thanks @smuzaffar! |
looks good. do you plan to change it more or should we merge it? |
We can merge it! :) |
It goes with the following job configuration: https://cmssdt.cern.ch/jenkins/job/cms-containers-run-cmssw-test/configure